Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blacken stragglers #6611

Merged
merged 3 commits into from
Jun 21, 2021
Merged

Blacken stragglers #6611

merged 3 commits into from
Jun 21, 2021

Conversation

levbishop
Copy link
Member

In #6361 we missed running black on the examples directory and on setup.py
This PR corrects that oversight and updates Makefile and tox.ini so that black and pylint are all run on the relevant files.
I think it would also be reasonable to run just black on the toplevel to avoid missing other future additions.

@levbishop levbishop added the type: qa Issues and PRs that relate to testing and code quality label Jun 20, 2021
@levbishop levbishop requested a review from mtreinish June 20, 2021 22:30
@levbishop levbishop requested a review from a team as a code owner June 20, 2021 22:30
Copy link
Member

@mtreinish mtreinish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One inline comment on the tox.ini. Also can you update the azure-pipelines.yml file too to include the extra paths.

tox.ini Outdated Show resolved Hide resolved
@levbishop
Copy link
Member Author

One inline comment on the tox.ini. Also can you update the azure-pipelines.yml file too to include the extra paths.

Done. VS Code flagged a lot of complaints about the azure-pipelines.yml not being schema-compliant, but I guess since things were working before, that is spurious?

@mtreinish
Copy link
Member

One inline comment on the tox.ini. Also can you update the azure-pipelines.yml file too to include the extra paths.

Done. VS Code flagged a lot of complaints about the azure-pipelines.yml not being schema-compliant, but I guess since things were working before, that is spurious?

Yeah, I'm not sure what it's using for a schema. They may point to issues or deprecations in the config file. But it all works as expected so nothing it's reporting is fatal.

@mtreinish mtreinish added automerge Changelog: None Do not include in changelog labels Jun 21, 2021
@mergify mergify bot merged commit 9e9ad8a into Qiskit:main Jun 21, 2021
@levbishop levbishop deleted the blacken_stragglers branch June 23, 2021 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog type: qa Issues and PRs that relate to testing and code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants